Skip to content

Feature/tools OTEL#7

Merged
dimetron merged 22 commits intomainfrom
feature/tools-refactoring
Jul 15, 2025
Merged

Feature/tools OTEL#7
dimetron merged 22 commits intomainfrom
feature/tools-refactoring

Conversation

@dimetron
Copy link
Copy Markdown
Collaborator

@dimetron dimetron commented Jul 10, 2025

  • added telemetry
  • security validations
  • structured logging
  • e2e tests

#5

dimetron added 6 commits July 8, 2025 15:28
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
- security validations
- structured logging
- e2e tests

Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
@dimetron dimetron requested a review from EItanya as a code owner July 10, 2025 01:58
- security validations
- structured logging
- e2e tests

Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
@dimetron dimetron changed the title Feature/tools refactoring Feature/tools OTEL Jul 10, 2025
@dimetron dimetron self-assigned this Jul 10, 2025
dimetron added 13 commits July 10, 2025 05:51
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Comment thread cmd/main.go Outdated
Comment thread internal/telemetry/tracing.go Outdated
Comment thread internal/config/config.go Outdated
Comment thread internal/config/config.go Outdated
Comment thread internal/cache/cache.go Outdated
Comment thread internal/telemetry/middleware.go
Comment thread internal/telemetry/middleware.go
// Common validation patterns
var (
// K8s resource name pattern (RFC 1123)
k8sNamePattern = regexp.MustCompile(`^[a-z0-9]([-a-z0-9]*[a-z0-9])?$`)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These 2 are identical?

}

// SanitizeInput sanitizes input strings by replacing potentially dangerous characters
func SanitizeInput(input string) string {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently only used in tests?

Comment thread internal/telemetry/tracing.go Outdated
dimetron added 2 commits July 12, 2025 15:05
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
Signed-off-by: Dmytro Rashko <dmitriy.rashko@amdocs.com>
@dimetron dimetron merged commit 4ab717c into main Jul 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants